Thank you for your work, @mouhannad-sh! Looks great.

Yes, let's add at least one basic test that asserts an explicit empty string body ("") and Content-Type: application/json. There's an existing test/rest-api/body.test.ts test suite that asserts different body scenarios, I'd suggest to add this one where you think it makes a logical sense.

The test performs a request with a certain options, and has a request handler in the mocks file that returns any request body as a response body.

Take a look at how test actions/assertions are written for more details, I'm sure it'd be straightforward. Let me know if you have any questions.

I've rebased your branch and will provide the tests, if you don't mind.

Hey, I've took this pull request as an opportunity to structure the body.test.ts test suite better. Feel free to take a look at how it's now, and at that new integration test I've added for a request with a Content-Type: application/json and no body. I've verified it fails without your change and passes with it.

Thanks a lot @kettanaito you're a life saver 🔥

Hey guys, I like this library, can you please release the current Master as we are stuck at #200 . Thanks so much!

@sumitj18s thanks for the feedback. The fix is released in 0.19.1. Make sure to follow the worker file update instructions in your browser's console.

Thanks @kettanaito a lot for such prompt release & response!